Skip to content

CNTRLPLANE-3206: add periodic e2e job for self-managed Azure v2#79876

Open
bryan-cox wants to merge 1 commit into
openshift:mainfrom
bryan-cox:CNTRLPLANE-3206
Open

CNTRLPLANE-3206: add periodic e2e job for self-managed Azure v2#79876
bryan-cox wants to merge 1 commit into
openshift:mainfrom
bryan-cox:CNTRLPLANE-3206

Conversation

@bryan-cox
Copy link
Copy Markdown
Member

@bryan-cox bryan-cox commented May 29, 2026 •
edited by atlassian Bot
Loading

Summary

  • Adds a daily periodic e2e job (e2e-azure-v2-self-managed) for self-managed Azure HCP on the release-5.0 branch
  • Runs at 6am UTC using the hypershift-azure-e2e-v2-self-managed workflow with hypershift-azure cluster profile
  • Catches regressions outside of PR presubmit runs for the self-managed Azure v2 test suite
  • Adds n1minor release reference (OCP 4.22 ci stream) needed by the workflow for upgrade image dependencies

Test plan

  • Verify generated Prow job config is correct via make jobs WHAT=openshift/hypershift
  • Rehearse the periodic job with /pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

🤖 Generated with Claude Code

Summary by CodeRabbit

This PR updates OpenShift CI configuration in the openshift/release repository to add a daily periodic e2e job for HyperShift that validates self-managed Azure v2 on the release-5.0 branch.

What changed in practical terms

  • Adds a new daily periodic e2e job e2e-azure-v2-self-managed to the HyperShift release-5.0 periodic config. The job runs the hypershift-azure-e2e-v2-self-managed workflow using the hypershift-azure cluster profile to exercise self-managed Azure v2 scenarios outside of PR presubmits.
  • Adds a releases.n1minor candidate pointing to OCP 4.22 on the CI stream (release:n1minor); this release reference is required by the workflow for upgrade-image dependencies.
  • Equivalent n1minor entry was added/updated in the 4.23 periodic config so the release-image mapping is available where the workflow expects it.
  • Configuration-only change: no source code or exported API changes.

Why this matters

  • Provides regular (daily) e2e validation of self-managed Azure v2 clusters for the 5.0 branch, increasing the likelihood of catching regressions that presubmits may not exercise.

Files affected

  • ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.0__periodics.yaml (new periodic job + n1minor)
  • ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.23__periodics.yaml (matching n1minor)

Verification / test plan

  • Verify generated Prow job config: make jobs WHAT=openshift/hypershift
  • Rehearse the periodic: /pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

Notes

  • openshift-ci-robot confirmed a valid JIRA reference (CNTRLPLANE-3206) but warned the referenced JIRA issue has a missing/invalid target version for the target branch (expected target version "5.0.0").

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label May 29, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented May 29, 2026 •
edited by atlassian Bot
Loading

@bryan-cox: This pull request references CNTRLPLANE-3206 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

  • Adds a daily periodic e2e job (e2e-azure-v2-self-managed) for self-managed Azure HCP on the release-5.0 branch
  • Runs at 6am UTC using the hypershift-azure-e2e-v2-self-managed workflow with hypershift-azure cluster profile
  • Catches regressions outside of PR presubmit runs for the self-managed Azure v2 test suite
  • Adds n1minor release reference (OCP 4.22 ci stream) needed by the workflow for upgrade image dependencies

Test plan

  • Verify generated Prow job config is correct via make jobs WHAT=openshift/hypershift
  • Rehearse the periodic job with /pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

🤖 Generated with Claude Code

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 29, 2026 •
edited
Loading

Walkthrough

Adds releases.n1minor (OCP 4.22 on ci) and a new periodic job e2e-azure-v2-self-managed (daily cron, hypershift-azure profile, OCP_IMAGE_N1: release:n1minor, env vars incl. HYPERSHIFT_AZURE_LOCATION: centralus) in HyperShift periodics for 4.23 and 5.0.

Changes

Hypershift Azure E2E Testing Configuration

Layer / File(s) Summary
Release candidate mapping
ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.23__periodics.yaml, ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.0__periodics.yaml		 Defines releases.n1minor as a new OCP release candidate targeting version "4.22" on the ci stream.
Periodic Azure E2E test job
ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.23__periodics.yaml, ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.0__periodics.yaml		 Adds e2e-azure-v2-self-managed periodic job (cron), configured with hypershift-azure cluster profile, OCP_IMAGE_N1: release:n1minor, hypershift-azure-e2e-v2-self-managed workflow, and env vars including ENABLE_HYPERSHIFT_CERT_ROTATION_SCALE and HYPERSHIFT_AZURE_LOCATION: centralus.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

lgtm, area/pipelines, rehearsals-ack

Suggested reviewers

  • enxebre
  • muraee
🚥 Pre-merge checks | ✅ 14 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Test Structure And Quality ❓ Inconclusive This PR modifies YAML CI configuration files, not Ginkgo test code. The check requires reviewing Ginkgo test code for quality patterns, but none exists in this PR. The custom check is inapplicable to this PR. This is a CI configuration repository where the changes are YAML periodic test job definitions, not Ginkgo test implementations.
✅ Passed checks (14 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a periodic e2e job for self-managed Azure v2, which aligns with the additions to both HyperShift periodics configuration files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed PR contains no Ginkgo tests; added test files use standard Go testing package with testing.T, not Ginkgo BDD patterns (It/Describe/Context).
Microshift Test Compatibility ✅ Passed This PR modifies only CI configuration YAML files, not test code. No Ginkgo e2e test implementations are added; therefore the MicroShift compatibility check is not applicable.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PR only adds YAML CI configuration, not new Ginkgo test code. Check applies only when new tests (Go source with It/Describe/etc) are added.
Topology-Aware Scheduling Compatibility ✅ Passed PR modifies only CI job configuration files (ci-operator periodic test configs), not deployment manifests, operator code, or controllers. Check applies only to these three artifact types.
Ote Binary Stdout Contract ✅ Passed PR contains only YAML configuration file changes with no executable code; OTE Binary Stdout Contract check applies to code-level stdout writes which are not present.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed This PR adds CI configuration and workflow scheduling only, not new Ginkgo e2e test code. The custom check applies to new tests added; no test code is present in this commit.
No-Weak-Crypto ✅ Passed PR modifies only YAML CI/CD configuration files. No weak cryptography (MD5, SHA1, DES, RC4, 3DES, Blowfish, ECB), custom crypto implementations, or insecure secret comparisons detected.
Container-Privileges ✅ Passed PR adds CI operator config for periodic E2E tests; no container privilege escalation settings found in the added YAML files.
No-Sensitive-Data-In-Logs ✅ Passed No passwords, tokens, API keys, or PII found. Credentials handled securely with 'set +x' in scripts; all env vars are non-sensitive.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 29, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bryan-cox

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 29, 2026
@openshift-ci openshift-ci Bot requested review from enxebre and muraee May 29, 2026 15:41
@bryan-cox bryan-cox force-pushed the CNTRLPLANE-3206 branch 2 times, most recently from bd50d01 to 2daa697 Compare May 29, 2026 15:45
@bryan-cox @claude
CNTRLPLANE-3206: add periodic e2e job for self-managed Azure v2
e651005 
Add a daily periodic job (6am UTC) for the self-managed Azure v2 e2e
tests on the release-5.0 branch. This catches regressions outside of
PR presubmit runs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 1, 2026

[REHEARSALNOTIFIER]
@bryan-cox: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-aws-multi N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-ibmcloud-ovn-roks N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-ovn-conformance N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-openstack-aws-csi-cinder N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-openstack-aws-csi-manila N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aks-multi-x-ax N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-azure-kubevirt-ovn N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-aws-ovn-proxy-conformance N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-aks-ovn-conformance N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-aws-conformance-cilium N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-openstack-aws-conformance N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-multi N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-kubevirt-metal-ovn-disconnected N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-openstack-aws N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-ovn-conformance-ccm-techpreview N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-kubevirt-aws-ovn N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-kubevirt-ovn N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-upgrade N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-aws-external-oidc N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-aws-upgrade N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-ibmcloud-ovn-iks N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-ovn N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-kubevirt-metal-ovn-disconnected N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-aws-ovn-conformance-ccm N/A periodic Ci-operator config changed
periodic-ci-openshift-hypershift-release-4.23-periodics-e2e-openstack-aws-conformance N/A periodic Ci-operator config changed

A total of 61 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs.

A full list of affected jobs can be found here

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@bryan-cox
Copy link
Copy Markdown
Member Author

bryan-cox commented Jun 1, 2026

/pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 1, 2026

@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@bryan-cox
Copy link
Copy Markdown
Member Author

bryan-cox commented Jun 2, 2026

/pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 2, 2026

@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 2, 2026

@bryan-cox: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed e651005 link unknown /pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@bryan-cox bryan-cox mentioned this pull request Jun 2, 2026
Merged
2 tasks
@bryan-cox
Copy link
Copy Markdown
Member Author

bryan-cox commented Jun 2, 2026

/pj-rehearse periodic-ci-openshift-hypershift-release-5.0-e2e-azure-v2-self-managed

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 2, 2026

@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 2, 2026

@bryan-cox: job(s): periodic-ci-openshift-hypershift-release-5.0-e2e-azure-v2-self-managed either don't exist or were not found to be affected, and cannot be rehearsed

@bryan-cox
Copy link
Copy Markdown
Member Author

bryan-cox commented Jun 2, 2026

/pj-rehearse periodic-ci-openshift-hypershift-release-5.0-periodics-e2e-azure-v2-self-managed

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot Bot commented Jun 2, 2026

@bryan-cox: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@enxebre enxebre Awaiting requested review from enxebre

@muraee muraee Awaiting requested review from muraee

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bryan-cox @openshift-ci-robot